-
Notifications
You must be signed in to change notification settings - Fork 5.9k
chore: attempt to fix docker #5106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
✨ code-server docs for PR #5106 is ready! It will be updated on every commit.
|
Codecov Report
@@ Coverage Diff @@
## main #5106 +/- ##
=======================================
Coverage 71.33% 71.33%
=======================================
Files 30 30
Lines 1685 1685
Branches 374 374
=======================================
Hits 1202 1202
Misses 413 413
Partials 70 70 Continue to review full report at Codecov.
|
e3b5e72
to
2380941
Compare
@@ -24,6 +24,7 @@ | |||
"package": "./ci/build/build-packages.sh", | |||
"postinstall": "./ci/dev/postinstall.sh", | |||
"publish:npm": "./ci/steps/publish-npm.sh", | |||
"publish:docker": "./ci/steps/docker-buildx-push.sh", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for consistency, i wanted it to be like the publish:npm
flow/script.
* chore: attempt to fix docker * Update .github/workflows/docker.yaml Co-authored-by: Asher <[email protected]> * chore: add publish:docker to scripts Co-authored-by: Asher <[email protected]>
We had an issue publishing v4.0.3 where the action couldn't find the artifacts to download. This changes that flow to download the release assets from the release for the Docker workflow.